content.js ➔ ... ➔ ??? - Code Metrics - Inspection of "Updated Linter & Rules" - prometheas/jira-2-omnifocus - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Pull Request — master (#17)

unknown

created 2017-08-26 16:11 UTC

content.js ➔ ... ➔ ??? A

↳ Parent: content.js ➔ ???

Complexity

Conditions	1
Paths	1

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	1
nc	1
nop	1
dl	0
loc	9
rs	9.6666
c	0
b	0
f	0

/* eslint-env browser, jquery, webextensions */

const $sendButton = $('<a id="send-to-omnifocus">Send to OmniFocus &raquo;</a>');

$(() => {
  // creates an OF task using the supplied ticket info
  function createTaskForTicket(ticket) {

    const message = {
      method: 'createTask',
      params: {
        name: `[${ticket.key}] ${ticket.summary}`,
        note: `${window.location.href}\n\n${ticket.description}`,
      },
    };

    chrome.runtime.sendMessage(message, (data) => {

      $('<iframe style="display:none">')
        .attr('src', data.url)
        .appendTo('body');
    });
  }

  $sendButton.prependTo('#viewissuesidebar');

  $sendButton.on('click', (evt) => {
    evt.preventDefault();

    createTaskForTicket({
      key: $('#key-val').text(),
      summary: $('#summary-val').text().replace(/\s+/, ' ').replace(/^\s*(\S.+\S)\s*$/, '$1'),
      description: $('#description-val').html(),
    });
  });
});


1			/* eslint-env browser, jquery, webextensions */
2
3			const $sendButton = $('<a id="send-to-omnifocus">Send to OmniFocus »</a>');
4
5			$(() => {
6			// creates an OF task using the supplied ticket info
7			function createTaskForTicket(ticket) {
			0 ignored issues – show Bug introduced 2017-08-22 22:58 UTC by Report Bug Copy Issue Report The function `createTaskForTicket` is declared conditionally. This is not supported by all runtimes. Consider moving it to root scope or using `var createTaskForTicket ...nction() { /* ... */ };` instead. Loading history...
8			const message = {
9			method: 'createTask',
10			params: {
11			name: `[${ticket.key}] ${ticket.summary}`,
12			note: `${window.location.href}\n\n${ticket.description}`,
13			},
14			};
15
16			chrome.runtime.sendMessage(message, (data) => {
			0 ignored issues – show Bug introduced 2017-08-22 22:58 UTC by Report Bug Copy Issue Report The variable `chrome` seems to be never declared. If this is a global, consider adding a `/** global: chrome */` comment. This checks looks for references to variables that have not been declared. This is most likey a typographical error or a variable has been renamed. To learn more about declaring variables in Javascript, see the MDN. Loading history...
17			$('<iframe style="display:none">')
18			.attr('src', data.url)
19			.appendTo('body');
20			});
21			}
22
23			$sendButton.prependTo('#viewissuesidebar');
24
25			$sendButton.on('click', (evt) => {
26			evt.preventDefault();
27
28			createTaskForTicket({
29			key: $('#key-val').text(),
30			summary: $('#summary-val').text().replace(/\s+/, ' ').replace(/^\s(\S.+\S)\s$/, '$1'),
31			description: $('#description-val').html(),
32			});
33			});
34			});
35

prometheas / jira-2-omnifocus

Pull Request — master (#17)

content.js ➔ ... ➔ ??? A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like